Fix debug build#1119
Conversation
301b3ee to
58ac112
Compare
In the dynamic build, crt1_shared.o calls lind_debug_printf (when compiled with LIND_DEBUG), but the shared libc didn't export it. wasm-ld with --allow-undefined turned it into an unresolved import, causing "unknown import: env::lind_debug_printf" at runtime. Add --export-if-defined=lind_debug_printf to the shared libc link.
58ac112 to
20732d3
Compare
End-to-End Test ReportTest PreviewUnified Test Report grate harness
Cases
static harnessTest ReportDeterministic TestsSummary
Test Results by Category
Fail TestsSummary
wasm harnessTest ReportDeterministic TestsSummary
Test Results by Category
Fail TestsSummary
Test Results by Category
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
qianxichen233
left a comment
There was a problem hiding this comment.
previously a commit accidentally deleted all the debug related functions in lind_debug.c: 0f386b3, specifically, lind_debug_num and lind_debug_import are all gone
Later for some reason, lind_debug_import in crt1 is replaced into lind_debug_printf, which is also not an expected change: 4265012
All of these changes need to revert back and this will naturally fix the issue
|
ok @qianxichen233 can @DanielZ224 fix this today? I think this is mildy urgent |
|
@rennergade I'll take a look and try to fix it today! |
|
Hi @rennergade , I pushed a branch with a tentative fix here:
Summary of changes:
I also ran the relevant build/check command locally and it completed without errors. Could you take a quick look and let me know whether this direction looks right? If it looks good, I’ll open a PR from this branch. |
Closes #1118
Needed to add lind_debug_printf to shared script.